Skip to content

Respect configurable toggle key modifier#1

Closed
Sean-Kenneth-Doherty wants to merge 1 commit into
NathanKell:masterfrom
Sean-Kenneth-Doherty:settings-toggle-fallbacks
Closed

Respect configurable toggle key modifier#1
Sean-Kenneth-Doherty wants to merge 1 commit into
NathanKell:masterfrom
Sean-Kenneth-Doherty:settings-toggle-fallbacks

Conversation

@Sean-Kenneth-Doherty
Copy link
Copy Markdown

@Sean-Kenneth-Doherty Sean-Kenneth-Doherty commented May 17, 2026

Summary

  • Respect the existing useModifier setting when toggling the AeroGUI window.
  • Keep default settings if the AEROGUI config node is missing.
  • Ignore invalid key values instead of failing startup.

Validation

  • git diff --check
  • checked GameData/AeroGUI/AeroGUISettings.cfg brace balance
  • direct source compile against local KSP 1.12.5 managed assemblies; compile succeeded with three existing unused-variable warnings

No runtime KSP validation was performed.

@Sean-Kenneth-Doherty
Copy link
Copy Markdown
Author

Closing this under my tightened quality bar rather than leaving an unrequested input/config behavior draft in your queue. The branch fixes plausible toggle/config handling and source compilation passed, but there is no current issue, maintainer request, or in-game KSP proof of the window behavior. I will not reopen unless you want this exact AeroGUI fix or there is a concrete player-facing defect with proof.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant